Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests for tiled image map view controller + bugs + docs + zoom. #20

Merged
merged 7 commits into from
Mar 14, 2014
Merged

Tests for tiled image map view controller + bugs + docs + zoom. #20

merged 7 commits into from
Mar 14, 2014

Conversation

dblock
Copy link
Collaborator

@dblock dblock commented Mar 14, 2014

Fixed downloading of tiles that may happen multiple times. In a slow network tiles will take a long time to download, but you may get lots of repaint events. This will cause the queue to get more downloads for the same tiles already being downloaded. We now hold on to a map of URLs to downloads and make sure to only have a single download operation per url running.

Added tests for the tiled image view controller. Notice a missing tile in the snapshot image, this seems to be a bug in iOS (@orta, can you comment with some links, please?).

Zoom will now pan towards the point being tapped. This is really neat and works like any good map would :) All credit is @orta.

@orta
Copy link
Collaborator

orta commented Mar 14, 2014

Re:tiling occasionally missing spots: https://twitter.com/Cocoanetics/status/402690909567221760

@dblock
Copy link
Collaborator Author

dblock commented Mar 14, 2014

Thread says this is fixed in iOS 7.1.

@orta
Copy link
Collaborator

orta commented Mar 14, 2014

Yes, travis is not on 7.1 yet. 👍

neilang added a commit that referenced this pull request Mar 14, 2014
Tests for tiled image map view controller + bugs + docs + zoom.
@neilang neilang merged commit 3e93654 into neilang:version-3.0 Mar 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants